Skip to content
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

Added uwp sample for Todolist #62

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Added uwp sample for Todolist #62

wants to merge 2 commits into from

Conversation

AutumnEvans418
Copy link

@AutumnEvans418 AutumnEvans418 commented Jan 24, 2020 •
edited
Loading

Added a UWP sample for the todo sample:

image

The event log logger provider in Microsoft extensions is not implemented for UWP and will throw an exception if you run the app, so I updated the app to remove that logger:

  .ConfigureLogging(d =>
                {
                    d.ClearProviders();
                    d.AddConsole();
                })

This was the only change I had to make for UWP to work.

@dnfclas
Copy link

dnfclas commented Jan 24, 2020 •
edited
Loading

CLA assistant check
All CLA requirements met.

@AutumnEvans418 AutumnEvans418 requested a review from Eilon January 24, 2020 03:38
@Eilon
Copy link
Contributor

Eilon commented Jan 24, 2020

Very cool, I will take a look at this!

@AutumnEvans418 AutumnEvans418 mentioned this pull request Mar 13, 2020
Open
Base automatically changed from master to main March 16, 2021 18:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@Eilon Eilon Awaiting requested review from Eilon

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AutumnEvans418 @dnfclas @Eilon